-
Notifications
You must be signed in to change notification settings - Fork 12.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable] 1.52.1 release #85097
[stable] 1.52.1 release #85097
Conversation
@Mark-Simulacrum: no appropriate reviewer found, use r? to override |
|
This comment has been minimized.
This comment has been minimized.
c84bb38
to
5e94af7
Compare
This comment has been minimized.
This comment has been minimized.
5e94af7
to
c6b04af
Compare
This comment has been minimized.
This comment has been minimized.
This disables incremental (i.e., -Cincremental) taking effect unless an environment variable, RUSTC_FORCE_INCREMENTAL, is set to 1 in the environment of the running rustc. Currently incremental causes errors for many users, and we do not have an expectation of being able to quickly fix these errors in a backportable way - so, for now, disable incremental entirely. The user can still opt-in, but this way the majority of users merely get slower builds, not broken builds.
This bumps the stable version number and adds corresponding release notes.
c6b04af
to
a0190b5
Compare
@bors r+ rollup=never p=100 |
📌 Commit a0190b5 has been approved by |
Rust users **should** upgrade to 1.52.0 or 1.52.1: the bugs that are detected by | ||
newly added incremental verification are still present in past stable versions, | ||
and are not yet fixed on any channel. These bugs can lead to miscompilation of | ||
Rust binaries. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: I think it would be good to scope this sentence more tightly by moving the mention of incr comp from the next sentence up into this one. Ie: "Rust binaries when incremental compilation is enabled"
☀️ Test successful - checks-actions |
Note that this may not be the version we end up going with. However, in the interests of having the artifacts available should we choose to use them, this PR will prepare a set of artifacts which:
Note that the release notes mark Monday as the release date; I think it is likely that if we choose to go down this path we should indeed release on Monday, and potentially follow up in a week or two (e.g., May 20th) with a 1.52.2 if we have confidence in some collection of fixes. I think this will be unlikely, though.
Please also note that this PR breaks normal policy by landing commits/work only on the stable branch. It is my intent to follow up with PRs toward beta and master, as well, but in the interests of making sure we have artifacts as early as possible I am posting this first. It will also let us provide an ask for testing, via the dev-static bucket, sooner rather than later.